Skip to content

Clarify setup-required state before processing starts#49

Open
adityakulraj wants to merge 1 commit intosamhu1:masterfrom
adityakulraj:feature/adkunwar-issue-26-provider-setup
Open

Clarify setup-required state before processing starts#49
adityakulraj wants to merge 1 commit intosamhu1:masterfrom
adityakulraj:feature/adkunwar-issue-26-provider-setup

Conversation

@adityakulraj
Copy link
Copy Markdown
Contributor

Summary

  • add a new oagent:check-provider-setup IPC preflight to detect provider configuration issues before a turn starts
  • validate OpenRouter key presence and Ollama endpoint reachability (/api/tags probe with timeout)
  • expose the preflight via preload + renderer typings as window.clientCore.checkProviderSetup(...)
  • use this preflight in session send/revive paths so misconfigured runs produce a clear Setup required: ... system message instead of generic/stuck processing
  • include explicit fix guidance in messages: add OpenRouter key, start/fix Ollama, or switch provider

Validation

  • code-level verification of wiring and error flow in draft send + revive paths
  • local TypeScript/build checks could not be run in this environment (node_modules/.bin/tsc unavailable)

Closes #26

@adityakulraj adityakulraj requested a review from samhu1 as a code owner March 24, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify first-run setup when no model provider is configured

1 participant